Anonymize evaluation staff notes #2698 - #2721
Conversation
anjaton
changed the title
| course=cls.course, | ||
| name_de="Wie man Software testet", | ||
| name_en="Testing your software", | ||
| staff_notes="This evaluation has dropout answers", |
There was a problem hiding this comment.
I think this is a bit of a misleading note because this is not the case in this test, can we use something else?
|
@anjaton do you want to keep working on this, or should we take over / assign this issue to someone else? |
Co-authored-by: Niklas Mohrin <dev@niklasmohrin.de>
|
I thought it was finished. DO I have to accept the suggested changes? Sorry i didnt know that. If there is something left to do you can take over. |
|
The way it usually works is that we reviewers give some feedback comments that the PR author then incorporates into the PR (this can be done by just pushing more commits to the branch). The author can then click "resolve" on all these comments and re-request a review on Github in the reviewers list on the right. (For PRs from new contributors, we keep an eye out on them ourselves and check in every now and then) And at all times, you can reach out by writing a comment with any questions or problems. Once the reviewers are happy with the PR, we merge it so that the changes are also present in the main branch of the e-valution/EvaP repository and included in the next release; it will look like in your other PR: https://redirect.github.com/e-valuation/EvaP/pull/2700 :) So for this PR, there currently is one unresolved comment by me, which suggests that we change the value of |
3 participants
fixes #2698